Don't use an 8MB stack any more

Now that the resolver isn't recursive this shouldn't be necessary

Merge branch 'non-recursive-activate' of https://github.com/jyasskin/cargo

Auto merge of #1992 - carols10cents:cleanup, r=alexcrichton

Hiii! These are just a few small cleanup things that I took care of while working on #1991, but these should all be less controversial than that one is :)

Fixing typos and grammar in comments

Remove unnecessary PathSource update in cargo test

I don't think updating the source here is hurting anything, per se, it's
just unnecessary since ops::compile makes a PathSource and then calls
root_package, which will call update if it needs to.

Move test to a more appropriate location

This test isn't testing generate-lockfile at all-- it's generating a
lockfile with `\r\n` in it in the test itself and then testing that
building will succeed even if there are `\r\n`s in the lockfile.

Given that generate-lockfile could change and have no effect on this
test, I think this fits in better with the other `cargo build` tests.

Remove unneeded commented-out code

Make Dependency cheap to copy by having it store a refcounted inner object.

Auto merge of #1988 - alexcrichton:bump, r=brson

I'll tag 0.5.0 with the previous commit after this is merged.

Bump to 0.6.0

Auto merge of #1986 - mneumann:dragonfly-snapshot, r=alexcrichton

http://www.ntecs.de/downloads/rust/cargo-nightly-x86_64-unknown-dragonfly.tar.gz

This is the 0.5 version of cargo as it's much harder to build an old snapshot.

Add snapshot for DragonFly

http://www.ntecs.de/downloads/rust/cargo-nightly-x86_64-unknown-dragonfly.tar.gz

This is the 0.5 version of cargo as it's much harder to build an old snapshot.

Auto merge of #1949 - nicokoch:fix-1904, r=alexcrichton

This flag can be used to force cargo to run all tests, even if previous
tests failed. See #1904

Various small changes to cargo_test and friends.

- Reworked CargoTestError to contain Vec<ProcessError>
- Remove #[allow(trivial_casts)]
- Coding style fixes

Restructure cargo_test code and introduce CargoTestError

Auto merge of #1981 - alexcrichton:git-stamp-if-dead, r=brson

We already take this strategy for extracting tarballs from crates.io for
example, so this just applies the same strategy to checkouts of git repos.

Closes #1979

Use a stamp file to protect against corrupt checkouts

We already take this strategy for extracting tarballs from crates.io for
example, so this just applies the same strategy to checkouts of git repos.

Closes #1979

Fix another round of comments.

Auto merge of #1975 - alexcrichton:dont-capture-rustdoc, r=brson

We already started doing this for the compiler awhile back, so this just brings
rustdoc up to the same parity

Auto merge of #1972 - alexcrichton:update-curl, r=brson

Includes a fix for #1937 to not panic on downloading crates on newer OSX
versions.

Closes #1937

Don't capture rustdoc output for transitive deps

We already started doing this for the compiler awhile back, so this just brings
rustdoc up to the same parity

Auto merge of #1974 - alexcrichton:dont-document-build, r=brson

They're not actually relevant to the documentation, so omit them.

Don't document build dependencies

They're not actually relevant to the documentation, so omit them.

Auto merge of #1973 - alexcrichton:cargo-doc-release, r=brson

While this may not necessarily make sense all the time, this enables Cargo to
not rebuild anything if a `cargo build --release` was previously executed.

Add --release flag to `cargo doc`

While this may not necessarily make sense all the time, this enables Cargo to
not rebuild anything if a `cargo build --release` was previously executed.

Auto merge of #1969 - carols10cents:really-use-git, r=alexcrichton

Fixes #1210. If either `--vcs git` or `--vcs hg` is specified, assume
that means the user really wants the new project to use that VCS, even
if the project's location would be beneath an existing repository.

This was surprisingly easy! And would make me so happy-- my home dir is a git repo :)

Update curl bindings

Includes a fix for #1937 to not panic on downloading crates on newer OSX
versions.

Closes #1937

If --vcs flag specified, use vcs even if under an existing repo

Fixes #1210. If either `--vcs git` or `--vcs hg` is specified, assume
that means the user really wants the new project to use that VCS, even
if the project's location would be beneath an existing repository.

Round 2 of review:

* Converted pre-existing trace!s back from debug!
* Computed `cur` from the top-most frame instead of storing it.
* Wrapped to 80 columns.
* Removed prev_active and all_candidates from `BacktrackStack`
* Removed unnecessary type annotations.
* Called activation_error() only when starting to backtrack, since it bailed
  early when passed an error anyway.

Auto merge of #1966 - alexcrichton:fix-doc-again, r=huonw

It's possible to have multiple dependencies of the same name for a project if a
dependency shows up multiple times as a target-specific dependency. This means
that we can't just find the first one with the relevant name and assume it's the
right one, we instead need to check all of them.

Fix documenting target-specific dependencies for real

It's possible to have multiple dependencies of the same name for a project if a
dependency shows up multiple times as a target-specific dependency. This means
that we can't just find the first one with the relevant name and assume it's the
right one, we instead need to check all of them.

Auto merge of #1961 - alexcrichton:fix-doc-target-specific, r=huonw

The recent change in #1939 ended up accidentally pulling in irrelevant
platform-specific dependencies into a normal `cargo doc`, causing those builds
to fail. This change tweaks the logic for calculating documentation dependencies
to account for filtering based on the relevant platform.

Auto merge of #1964 - steveklabnik:doc_fix, r=alexcrichton

This has changed, apparently.

Fix syntax in build-script

This has changed, apparently.

Auto merge of #1963 - Frederick888:master, r=alexcrichton

The most common case is that if `cargo` is not installed to  `/usr` but somewhere else such as `/usr/local` (is this the default install prefix? don't remember...), user will see `cargo: command not found` after `sudo su` executed.

P.S. sorry for previous wrong PR... there was a redundant commit.

avoid warning when cargo not found

Auto merge of #1959 - lucab:lucab/skip-dotdir, r=alexcrichton

Cargo recursively looks for TOML manifest into child directories,
sometimes getting into unrelated files when exploring hidden
directories (eg. quilt .pc).

This commit lets cargo skip dot directories, to partially avoid
the issues described in #1423.

Signed-off-by: Luca Bruno <lucab@debian.org>

Only document platform-relevant dependencies

The recent change in #1939 ended up accidentally pulling in irrelevant
platform-specific dependencies into a normal `cargo doc`, causing those builds
to fail. This change tweaks the logic for calculating documentation dependencies
to account for filtering based on the relevant platform.

path: do not recurse into hidden/dot directories

Cargo recursively looks for TOML manifest into child directories,
sometimes getting into unrelated files when exploring hidden
directories (eg. quilt .pc).

This commit lets cargo skip dot directories, to partially avoid
the issues described in #1423.

Signed-off-by: Luca Bruno <lucab@debian.org>

Reset `parent`, `cur`, and `dep` when backtracking.

Fix @alexcrichton's comments.

Merge past #1939.

Auto merge of #1960 - alexcrichton:build-static, r=huonw

There's no real reason to eagerly link build scripts dynamically as they're not
going to benefit that much from dynamic linking. Most of the time they only have
one dynamic dependency, libstd, and most other Rust programs don't link it
dynamically so there's not really many space savings either.

Don't link build scripts dynamically

There's no real reason to eagerly link build scripts dynamically as they're not
going to benefit that much from dynamic linking. Most of the time they only have
one dynamic dependency, libstd, and most other Rust programs don't link it
dynamically so there's not really many space savings either.

Auto merge of #1939 - impl:resolve-deps-with-host-triple, r=alexcrichton

This fixes a specific issue with platform-specific dependencies of build dependencies when cross-compiling. The gist of the problem is that when dependencies are initially resolved, only the target platform is considered, which can cause the later analysis of the graph by the custom build process to miss dependencies.

I fixed the issue by including both target and host platforms in the dependency resolution process. I'm brand new to the Rust ecosystem and language, so I'm not sure if this is the right route architecturally, or if my code is even remotely acceptable. Any feedback would be greatly appreciated!

I've also included a test case to demonstrate the problem; it fails against the current master.

Thanks!

Auto merge of #1955 - carols10cents:even-more-manifest-path-consistency, r=alexcrichton

Hiii :) This builds on #1953, so if yinz like this better, I'll close that one.

This PR makes all commands that take `--manifest-path` (except locate-project, see below) behave in a consistent way with regards to invalid paths, and hopefully gives better error messages.

This DOES break some instances where commands were "working", ie, completing their command successfully, with some path that they decided was right, not necessarily exactly the path that was passed in though. I'm not sure how this impacts whatever backwards compatibility guarantees cargo has. Given that those cases weren't using a valid path to an existing Cargo.toml, hopefully no one is relying on that behavior?

So locate-project... I'm not sure what to do about that one, exactly... If we define `--manifest-path` as a path to a Cargo.toml, then uhhhh we've located the project, it's right where you told us it is ;) So I think we should change it to be `--search-path` or similar? I'm too sleepy to take care of that tonight, so I just left it out of the manifest path tests.

Use find_root_manifest_for_cwd for now-duped code in verify-project

Give better error messages for invalid manifest-paths

Inspired by https://github.com/rust-lang/cargo/pull/1182.
Fixes #1158, #1772, #1773.

Add tests for different kinds of invalid manifest paths

Make verify-project able to verify the current working dir project

All other commands use the Cargo.toml in the current directory if
--manifest-path is not specified; this makes verify-project consistent.

Add tests for different arguments given to verify-project

Right now, only passing --manifest-path a path to a Cargo.toml file
works, but I think not specifying a path and verifying the current
working directory, if it has a Cargo.toml, should work as well for
consistency with all the other commands that optionally take
--manifest-path.

Make read-manifest accept consistent manifest-path arguments

And also preserve the existing behavior of accepting an absolute path to
a directory containing a Cargo.toml for backwards compatibility.

Add tests for different arguments given to read-manifest

Right now, only passing --manifest-path an absolute path to the
parent directory of a Cargo.toml file works, but I think the other
tests should work as well for consistency with other commands that
optionally take --manifest-path.

Fix platform-specific dependency resolution for custom builds.

This change resolves dependencies against all possible platforms.
Activation of dependencies for a given target in the context of one
or more platforms is now wholly handled by cargo_rustc.

Make ::can_panic() return true on x86_64-pc-windows-msvc

Use internal error instead of ProcessError when multiple tests fail

Auto merge of #1943 - mseri:patch-1, r=alexcrichton

It maybe a tacky way, but I think that changing `util::without_prefix` is a bad approach. Maybe modify `process.build_command` could be another possible approach.

It could be worth to take more time to be sure to find the root of the problem. I am quite new to rust, if anybody can point out the right way of dealing with it, I will be happy to submit a new PR.

Fix for test on windows

Added test for issue #1942

Auto merge of #1931 - thirtythreeforty:printinfo, r=alexcrichton

Compare #1621.  Like that pull request, this one solves #984.

I have rewritten most of @pyfisch's logic to account for @alexcrichton's comments.  Specifically, this code will correctly handle adding/removing multiple versions of one package, as well as several packages with different sources, but the same name.

The output looks like this:

```
[georgev@desertvoice cargo]$ cargo update
    Updating registry `https://github.com/rust-lang/crates.io-index`
    Updating libc v0.1.8 -> v0.1.10
    Updating memchr v0.1.3 -> v0.1.5
    Updating num v0.1.26 -> v0.1.27
    Updating rand v0.3.9 -> v0.3.10
    Updating rustc-serialize v0.3.15 -> v0.3.16
```

Comments welcome.

r? @alexcrichton

Improved fix for the issue #1942

Print information about updated packages on cargo update

Closes #984.

Reformat lines with more than 100 characters

Add test for --no-fail-fast

Cleaner implementation for --no-fail-fast.

This version also works for doctests.

Implement --no-fail-fast flag for cargo test.

This flag can be used to force cargo to run all tests, even if previous
tests failed. See #1904

Auto merge of #1950 - mbrubeck:cargo_profile, r=alexcrichton

None

Fix docs for the CARGO_PROFILE variable

Auto merge of #1945 - Frederick888:master, r=alexcrichton

This BASH function will find available cargo configurations recursively from the project directory to root and provide options for `--target`.

check and get dir by $PWD for Windows builds under Cygwin

avoid wrong invoking from non-project directory

add forgotten local declarations

bash completion for targets

Fix for issue #1942 'cargo run fails if called from executable folder'

It maybe a hacky way but I think that changing `util::without_prefix` is a bad approach. Maybe modify `process.build_command` could be another possible approach.

Allow inlining and fix up comments.

Since the algorithm isn't recursive anymore, inlining is fine.

Clean up.

Eliminate recursion!

Extract the sibling activation function.

Move all recursion to activate_deps.

Remove an unnecessary lifetime parameter.

Make activation_error always return an error.

Add some type annotations to make it easier to read the resolver.

Auto merge of #1940 - steveklabnik:whoops, r=huonw

Thanks @huonw :sob:

Close some tags.

Thanks @huonw :sob:

Auto merge of #1936 - steveklabnik:gh1870, r=alexcrichton

Fixes #1870

Auto merge of #1934 - steveklabnik:gh1930, r=alexcrichton

Fixes #1930

Auto merge of #1935 - steveklabnik:gh1898, r=alexcrichton

Fixes #1898

Auto merge of #1938 - blaenk:environment-variables-fix, r=steveklabnik

#1845 seems to have forgotten to add the page to the DOCS variable in the makefile

make sure that environment-variables page is built

PR 1845 seems to have forgotten to add the page to the DOCS variable in
the makefile

Update guide to use crates.io dependencies

Fixes #1930

Add a note to the FAQ about * for library dependencies

Fixes #1870

Make note about small divergance from SemVer

Fixes #1898

Auto merge of #1845 - steveklabnik:env_vars, r=alexcrichton

These were spread out before, now they're in one place.

Fixes #1538

r? @alexcrichton

I'm also missing an entry on `CARGO_PROFILE`, what does it do?

Consolidate and document environment variables

These were spread out before, now they're in one place.

Fixes #1538

Auto merge of #1932 - steveklabnik:gh1767, r=alexcrichton

Fixes #1767

Auto merge of #1933 - steveklabnik:gh1855, r=alexcrichton

Fixes #1855

Clarify -- in cargo run help

Fixes #1855

Clarfiy that --{test,bench} select targets

Fixes #1767

Auto merge of #1928 - alexcrichton:update, r=alexcrichton

* Update dependencies that are getting warnings on nightly
* Tweak expected output of failing tests

Update for Rust nightly

* Update dependencies that are getting warnings on nightly
* Tweak expected output of failing tests